When do we need to bump the contract versions in deploy? For every time the contracts change or whenever we wanna deploy? I'm wary of using the versions in a yarn workspace since it makes local development harder, but wanna understand more first.

When do we need to bump the contract versions in deploy? For every time the contracts change or whenever we wanna deploy?

we need to bump the contract versions when we release a new tagged version of the contracts, which will happen each time we perform a protocol sprint + audit + on-chain upgrade. this will be approx every 6-8 weeks.

right now, if we make protocol changes during a protocol sprint, then deploy a new chain, the contracts on the new chain will be un-audited & inconsistent with the other chains' contracts

Instead of hardcoding it in the package.json, could the deployment script fetch the latest npm versions from the npm registry?

putting it in the package.json will also cause problems when we want to deploy test contracts that aren't tagged on EVM. e.g. testing in local env?

running into rough edges on to my understanding of JS packaging

@anna-carroll because we may need to run deployments from commits older than this, you'll need to add "check the version" to your list of manual deploy pre-flight checks 😓

because we may need to run deployments from commits older than this

the catch is that the deploy script itself needs to be in sync with the version of the contracts it's deploying.
changes to contract interface affect deploy 😓

i we're deploying workspace^ contracts, then also workspace^ deploy script.
else, need tagged contract version + tagged deploy script..

Okay I have created a ticket where we should discuss this: #325

Closing PR until we have aligned on an approach

i we're deploying workspace^ contracts, then also workspace^ deploy script. else, need tagged contract version + tagged deploy script..

ya, for any given commit, CI should guarantee that the version of deploy in that commit is compatible with its dependency. So when we tag a contract release, we need to bump the deploy script dependency version. If we always do that correctly, we should keep things in sync I guess? 🤔